UI changes#243
Conversation
…es as in original)
|
@Lalitya31 is attempting to deploy a commit to the komalsony234-1530's projects Team on Vercel. A member of the Team first needs to authorize it. |
|
@komalharshita resolved merge conflicts , ready for review |
|
@Lalitya31 Thanks for the detailed UI contribution and the effort put into improving the visual hierarchy and overall polish of the project. There are several thoughtful ideas here, especially around spacing, typography, and information presentation. However, this PR is not ready for approval yet for a few reasons:
Please:
Once conflicts are resolved and the scope is refined, this can be reviewed again. |
|
@komalharshita I have resolved conflicts twice, yes I will do it again. Since perspective varies I cannot decide what changes are "unnecessary" because from my experience of design all of them seemed relevant to me for being a balanced visual identity (yes I do follow UX philosophies a bit rigidly sometimes) so could you please specify what changes do I need to revert? I'd follow up with your requirements accordingly. |
|
Closing this PR |
Summary
Improvised :
Related Issue
Closes #44
Type of Change [required]
data/projects.jsonScreenshots (if UI change)
Before :



AFTER :




How to Test This PR [required]
git checkout uipip install -r requirements.txtpython3 -m http.server 800Self-Review Checklist [required]
feat/,fix/,docs/,data/,style/,test/python tests/test_basic.pyand all 27 tests passflake8 .locally and there are no errorsprint()orconsole.log()debug statements